Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WebHost: noscript faq and glossary #4061

Merged
merged 6 commits into from
Oct 16, 2024
Merged

Conversation

Berserker66
Copy link
Member

What is this fixing or adding?

Makes /faq and /glossary work with noscript/uMatrix and similar, by not using JS anymore and rendering the md to html on server.

How was this tested?

locally

If this makes graphical changes, please attach screenshots.

I did not see a difference, which is the point.

@github-actions github-actions bot added affects: webhost Issues/PRs that touch webhost and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Oct 16, 2024
@Berserker66 Berserker66 changed the title Webhost noscript faq and glossary WebHost: noscript faq and glossary Oct 16, 2024
Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bullets in glossary -> world do not render correctly

@@ -9,3 +9,4 @@ bokeh>=3.1.1; python_version <= '3.8'
bokeh>=3.4.3; python_version == '3.9'
bokeh>=3.5.2; python_version >= '3.10'
markupsafe>=2.1.5
Markdown>=3.7
Copy link
Member

@black-sliver black-sliver Oct 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also missing newline at end of file here

@Berserker66 Berserker66 added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Oct 16, 2024
@Berserker66
Copy link
Member Author

Adding bugfix tag as the pages don't work for some users currently.

@black-sliver
Copy link
Member

Hm, it now renders correctly, but there is still a slight difference in the DOM. I'm gonna have one more looksie

Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sane lists are not sane after all.

With just mdx_breakless_lists it produces a DOM very similar to github and the old js code.

WebHostLib/misc.py Outdated Show resolved Hide resolved
WebHostLib/misc.py Outdated Show resolved Hide resolved
WebHostLib/requirements.txt Outdated Show resolved Hide resolved
Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Berserker66 Berserker66 merged commit 375b579 into main Oct 16, 2024
30 checks passed
@Berserker66 Berserker66 deleted the webhost_noscript_faq_and_glossary branch October 16, 2024 21:28
@LegendaryLinux
Copy link
Member

Do these pages still auto-scroll to the #target-div if provided in the URL?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: webhost Issues/PRs that touch webhost and may need additional validation. is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants